Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
에러 핸들링을 위해 try, catch를 사용하신 점은 좋습니다.
다만, 적절한 곳에서 에러 핸들링이 가능하도록 하는 것도 중요한데요.
(가령 React에서는 Promise를 던지는 경우 Suspense에서 위임하고, 오류를 던지는 경우 ErrorBoundary에 위임할 수 있습니다.)
API 요청 함수는 서버 상태를 관리하는 Tanstack Query의 useQuery, useMutation 등에 위임하여 처리할 수 있도록 외부에 오류를 전달해 주시는게 좋을 것 같습니다. :)
(그리고 catch 구문 내에서 console.error의 결과를 반환하시는데, 이 경우 undefined가 반환됩니다.)
| folderId: selectCategoryId | ||
| }), | ||
| onSuccess: () => { | ||
| refetch(); |
There was a problem hiding this comment.
Query Invalidation이 가능하면 refetch 대신 사용하시는 걸 권장드립니다~!!
There was a problem hiding this comment.
enabled 설정하면 invalidateQueries 사용못한다고 알고있는데 맞나요??
There was a problem hiding this comment.
enabled=false인 useQuery 말씀하시는건가요?
이렇게 사용한 경우는 없어서, 확인해봐야 알 것 같네요. (데이터가 있는데 enabled=false인 경우, active 쿼리에 해당하는지를 잘 모르겠네요.)
queryClient.invalidateQueries의 옵션에서 refetchType을 사용하시거나,
꼭 refetch를 하셔야 된다면 queryClient.refetchQueries를 쓰셔도 될 것 같습니다. (Props drilling으로 refetch를 전달할 필요는 없어 보입니다~!!)
|
Optimistic Update 작성하신 방식은 크게 문제는 없어 보입니다. 현재 콘솔 창에 403 Forbidden 오류 응답이 들어오는데, 네트워크 탭에서 요청 전달 시 문제가 없었는지 확인해 보시는게 좋을 것 같습니다. |
|
요청 주신 내용 위주로 리뷰 드렸습니다. |
hoody-jellybean
merged commit 0129604
into
codeit-bootcamp-frontend:part3-이우혁
영상에서는 OnError 처리한 것도 포함시키려고 api url에 아무 문자열 추가해서 테스트하였습니다! |
요구사항
기본
심화
주요 변경사항
멘토에게
2024-05-21.6.23.00.mov